Skip to content

feat: Add Debian-specific information to LVGL OOB docs#201

Merged
StaticRocket merged 1 commit into
TexasInstruments:masterfrom
jsuhaas22:debianize-lvgl-docs
Jun 13, 2025
Merged

feat: Add Debian-specific information to LVGL OOB docs#201
StaticRocket merged 1 commit into
TexasInstruments:masterfrom
jsuhaas22:debianize-lvgl-docs

Conversation

@jsuhaas22

Copy link
Copy Markdown
Collaborator

Presently, the LVGL OOB page is written under the assumption that the user is using Yocto. For Debian, some of the lines have to be changed. Therefore make those changes.

Also add the doc in Debian's tree.

VeeruPrudhvi
VeeruPrudhvi previously approved these changes Mar 4, 2025

@VeeruPrudhvi VeeruPrudhvi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Comment thread source/system/Demo_User_Guides/TI_LVGL_Demo_User_Guide.rst Outdated
Comment thread source/system/Demo_User_Guides/TI_LVGL_Demo_User_Guide.rst Outdated
Comment thread source/system/Demo_User_Guides/TI_LVGL_Demo_User_Guide.rst Outdated
Comment thread source/system/Demo_User_Guides/TI_LVGL_Demo_User_Guide.rst Outdated
Comment thread source/system/Demo_User_Guides/TI_LVGL_Demo_User_Guide.rst Outdated
Comment thread source/system/Demo_User_Guides/TI_LVGL_Demo_User_Guide.rst Outdated
Comment thread source/system/Demo_User_Guides/TI_LVGL_Demo_User_Guide.rst Outdated
Comment thread source/system/Demo_User_Guides/TI_LVGL_Demo_User_Guide.rst Outdated
Comment thread source/system/Demo_User_Guides/TI_LVGL_Demo_User_Guide.rst
@cshilwant

Copy link
Copy Markdown
Member

@jsuhaas22 is this PR still valid?
If yes, please plan to address the review comments

@cshilwant cshilwant left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Resolve merge conflicts

@jsuhaas22 jsuhaas22 force-pushed the debianize-lvgl-docs branch from 545dc2e to 6575258 Compare May 16, 2025 04:25
Comment thread configs/AM62LX/AM62LX_debian_toc.txt Outdated
Comment thread source/debian/Demo_User_Guides/index_Demos.rst Outdated
Comment thread source/linux/Demo_User_Guides/TI_LVGL_Demo_User_Guide.rst
Comment thread source/linux/Demo_User_Guides/TI_LVGL_Demo_User_Guide.rst Outdated
Comment thread source/linux/Demo_User_Guides/TI_LVGL_Demo_User_Guide.rst Outdated
Comment thread source/linux/Demo_User_Guides/TI_LVGL_Demo_User_Guide.rst
Comment thread source/linux/Demo_User_Guides/TI_LVGL_Demo_User_Guide.rst
@jsuhaas22 jsuhaas22 force-pushed the debianize-lvgl-docs branch from 6575258 to 765197c Compare June 3, 2025 10:27
@VeeruPrudhvi

Copy link
Copy Markdown
Contributor

@StaticRocket and @cshilwant ,

Can you please review the changes.

Comment thread source/linux/Demo_User_Guides/TI_LVGL_Demo_User_Guide.rst
Comment thread source/linux/Demo_User_Guides/TI_LVGL_Demo_User_Guide.rst Outdated
Comment thread source/linux/Demo_User_Guides/TI_LVGL_Demo_User_Guide.rst
@jsuhaas22 jsuhaas22 force-pushed the debianize-lvgl-docs branch from 765197c to 3cfff79 Compare June 10, 2025 10:15
Presently, the LVGL OOB page is written under the assumption that the
user is using Yocto. For Debian, some of the lines have to be changed.
Therefore make those changes.

Also add the doc in Debian's tree.

Signed-off-by: Suhaas Joshi <s-joshi@ti.com>
@jsuhaas22 jsuhaas22 force-pushed the debianize-lvgl-docs branch from 3cfff79 to 12e53d6 Compare June 10, 2025 10:30
@praneethbajjuri

Copy link
Copy Markdown
Collaborator

@StaticRocket and @cshilwant ,

Can you please review the changes.

@VeeruPrudhvi , @jsuhaas22 the review comments are still open. please get them addressed for the PR to be in a shape of integration.

@StaticRocket StaticRocket merged commit 350e00e into TexasInstruments:master Jun 13, 2025
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants